overview: Don't fade OverviewActor

Now that the workspace display is properly allocated for each frame,
we don't want to fade the entire overview anymore.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1624>

diff --git a/js/ui/overview.js b/js/ui/overview.js
index aabc4ec07606682c49e5bfa7939237564aacdf21..6606db594dc57d82d18c73660bb8806261ead103 100644 (file)
--- a/js/ui/overview.js
+++ b/js/ui/overview.js
@@ -98,32 +98,11 @@ class OverviewActor extends St.BoxLayout {
     }
 
     animateToOverview(callback) {
-        this._controls.animateToOverview();
-
-        this.opacity = 0;
-        this.ease({
-            opacity: 255,
-            mode: Clutter.AnimationMode.EASE_OUT_QUAD,
-            duration: ANIMATION_TIME,
-            onStopped: () => {
-                if (callback)
-                    callback();
-            },
-        });
+        this._controls.animateToOverview(callback);
     }
 
     animateFromOverview(callback) {
-        this._controls.animateFromOverview();
-
-        this.ease({
-            opacity: 0,
-            mode: Clutter.AnimationMode.EASE_OUT_QUAD,
-            duration: ANIMATION_TIME,
-            onStopped: () => {
-                if (callback)
-                    callback();
-            },
-        });
+        this._controls.animateFromOverview(callback);
     }
 
     get dash() {

diff --git a/js/ui/overviewControls.js b/js/ui/overviewControls.js
index 516de21b6f59cab24beb3ee83755722c5e1c98b7..756943630caca6ab36bfe80837d21e67f055f508 100644 (file)
--- a/js/ui/overviewControls.js
+++ b/js/ui/overviewControls.js
@@ -177,17 +177,21 @@ class ControlsManager extends St.Widget {
         this._workspaceAdjustment.value = activeIndex;
     }
 
-    animateToOverview() {
+    animateToOverview(callback) {
         this.viewSelector.animateToOverview();
 
         this._stateAdjustment.value = ControlsState.HIDDEN;
         this._stateAdjustment.ease(ControlsState.WINDOW_PICKER, {
             duration: Overview.ANIMATION_TIME,
             mode: Clutter.AnimationMode.EASE_OUT_QUAD,
+            onStopped: () => {
+                if (callback)
+                    callback();
+            },
         });
     }
 
-    animateFromOverview() {
+    animateFromOverview(callback) {
         this._ignoreShowAppsButtonToggle = true;
 
         this.viewSelector.animateFromOverview();
@@ -198,6 +202,9 @@ class ControlsManager extends St.Widget {
             onStopped: () => {
                 this.dash.showAppsButton.checked = false;
                 this._ignoreShowAppsButtonToggle = false;
+
+                if (callback)
+                    callback();
             },
         });
     }